Skip to content

update fashionscape to 1.4.2#12369

Merged
runelite-github-app[bot] merged 3 commits into
runelite:masterfrom
equirs:fashionscape-revamp-p1
Jun 21, 2026
Merged

update fashionscape to 1.4.2#12369
runelite-github-app[bot] merged 3 commits into
runelite:masterfrom
equirs:fashionscape-revamp-p1

Conversation

@equirs

@equirs equirs commented Jun 2, 2026

Copy link
Copy Markdown
Contributor

so I refactored all of the core stuff in my plugin several months ago but stupidly did it all in one massive commit, which (understandably) nobody wanted to review. I still want to get my changes out somehow but have been torn on how to do that exactly. What I'm hoping to do now is introduce most of the new classes in my refactor as dead code for now, and then I'll integrate them incrementally once they're all added.

@runelite-github-app

runelite-github-app Bot commented Jun 2, 2026

Copy link
Copy Markdown

@runelite-github-app

runelite-github-app Bot commented Jun 2, 2026

Copy link
Copy Markdown

This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed.


Internal use only: Reviewer details Maintainer details

@riktenx

riktenx commented Jun 20, 2026

Copy link
Copy Markdown
Member

use of serializationutils stuff is not allowed

@riktenx riktenx added the waiting for author waiting for the pr author to make changes or respond to questions label Jun 20, 2026
@runelite-github-app runelite-github-app Bot removed the waiting for author waiting for the pr author to make changes or respond to questions label Jun 20, 2026
@runelite-github-app runelite-github-app Bot merged commit 562fa58 into runelite:master Jun 21, 2026
2 of 3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants